Malicious site protection App settings #3734
Open
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Task/Issue URL: https://app.asana.com/0/72649045549333/1207944134334660/f
Tech Design URL: https://app.asana.com/0/72649045549333/1207273224076495/f
CC: @not-a-rootkit
Description:
Adds the Malicious Site Protection Settings in the General App Settings.
Steps to test this PR:
Prerequisites: Ensure that
MaliciousSiteProtectionFeatureFlags.swift
->isMaliciousSiteProtectionEnabled
andshouldDetectMaliciousThreat(forDomain domain: String?)
returntrue
.Scenario 1 - Malicious Site Protection section should not show in settings
MaliciousSiteProtectionFeatureFlags.swift
->isMaliciousSiteProtectionEnabled
returnsfalse
Scenario 2 - Malicious Site Protection section shows in settings
MaliciousSiteProtectionFeatureFlags.swift
->isMaliciousSiteProtectionEnabled
returnstrue
Scenario 3 - Disabling Malicious Site Protection does not show special error pages
MaliciousSiteProtectionFeatureFlags.swift
->isMaliciousSiteProtectionEnabled
andshouldDetectMaliciousThreat(forDomain domain: String?)
returntrue
.Scenario 4 - Enabling Malicious Site Protection shows special error pages
MaliciousSiteProtectionFeatureFlags.swift
->isMaliciousSiteProtectionEnabled
andshouldDetectMaliciousThreat(forDomain domain: String?)
returntrue
.Scenario 5 - Malicious Site Protection preference is remembered across app launches
Scenario 6 - Malicious Site Protection Learn More opens a tab
Scenario 7 - Disabling Malicious Site Protection shows a warning message to the user
Definition of Done (Internal Only):
Copy Testing:
’
rather than'
Orientation Testing:
Device Testing:
OS Testing:
Theme Testing:
Internal references:
Software Engineering Expectations
Technical Design Template